Skip to content

fix(docs): correct DataMorph install claims — package not on PyPI#46

Open
Coding-Dev-Tools wants to merge 1 commit into
masterfrom
cowork/improve-datamorph-v2
Open

fix(docs): correct DataMorph install claims — package not on PyPI#46
Coding-Dev-Tools wants to merge 1 commit into
masterfrom
cowork/improve-datamorph-v2

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

Corrects README install instructions that claimed the package is published to PyPI. DataMorph is only installable via git+https.\n\nCommit: c61748f

…d to PyPI

The README/AGENTS advertised `pip install datamorph-cli` (PyPI badge) but the
package is not actually published to PyPI (only `git+https` install works; the
coding-dev-tools.github.io/pypi-index 404s). Replace the false PyPI badge with a
"not published" badge and point the install instruction at the working
`git+https://github.com/Coding-Dev-Tools/datamorph.git` install. Ported from the
stranded local commit 1dbb981 (originally on the now-merged PR #45 branch) onto
current master as a fresh branch.

Co-Authored-By: Hermes Senior Dev Rotation <noreply@hermes-agent>
@github-actions

Copy link
Copy Markdown

🤖 Automated Code Review

✅ Ruff Lint — No issues

⚠️ Ruff Format — Formatting needed

Would reformat: conftest.py
Would reformat: src/datamorph/cli.py
Would reformat: src/datamorph/converters.py
Would reformat: tests/test_cli_error_paths.py
Would reformat: tests/test_converters.py
Would reformat: tests/test_edge_cases.py
Would reformat: tests/test_validate.py
7 files would be reformatted, 3 files already formatted

✅ Secret Detection — Clean

✅ Large Files — Within limits

📊 Diff Stats — 2 file(s) changed

 AGENTS.md | 2 +-
 README.md | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

Verdict: ⚠️ Warnings — Lint/format issues found. Recommend fixing before merge.

Automated by Coding-Dev-Tools/.github reusable workflow.

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner Author

⚖️ Council Gate Verdict — APPROVE

Risk level: low (documentation-only change; no auth/security/credential surface)
Decision: APPROVE
Agreement: 1.00
Session: council-72353c97-ce56-4ded-bf5d-a3032f26d679

Per-model scores

Model Overall Tests Safety Recommendation
tencent/hy3:free 5.0 5.0 5.0 approve
nvidia/nemotron-3-ultra-550b-a55b 5.0 5.0 5.0 approve
nvidia/llama-3.3-nemotron-super-49b-v1.5 5.0 5.0 5.0 approve
nvidia/nemotron-3-super-120b-a12b 5.0 5.0 5.0 approve

Aggregate metrics: avg_overall 5.00 · avg_safety 5.00 · avg_tests 5.00 · agreement 1.00

Summary

Documentation-only fix correcting inaccurate PyPI install claims in AGENTS.md and README.md. DataMorph is only installable via git+https; the change swaps the false pip install datamorph-cli / PyPI badge for the accurate git install instruction and a "not published" badge. All four responding council members recommend approve with unanimous 5.0 scores across correctness, safety, style, tests, and complexity. No code, test, or dependency changes; no security concerns.

Quorum note: several providers were unavailable this run (nvidia key missing, z-ai/openrouter 429/rate-limited, free-tier timeouts). 4 usable reviews satisfy quorum ≥2; verdict is valid.

Engraphis ref: (council Engraphis persistence unavailable this run) · workspace hermes-council

Automated Council Gate PR review · risk_level=low

@Coding-Dev-Tools Coding-Dev-Tools added the council-approved Council gate approved this PR (APPROVE / APPROVE_WITH_NITS) label Jul 17, 2026
@Coding-Dev-Tools

Copy link
Copy Markdown
Owner Author

Council Merge Gate — clean APPROVE, still policy-BLOCKED

  • Run council-merge-gate-20260717T024344Z, session council-13d3a5b8-d5e0-4e5b-b32d-8ca2236dc3e8
  • Verdict: APPROVE (quorum=4). Ground-truth: mergeable=MERGEABLE, CI green, but mergeStateStatus=BLOCKED.
  • Merge failed: base branch policy prohibits the merge.
  • Action required: a human approving review (bot approvals are label/comment only, not a GitHub review).
  • Left OPEN — no --admin.

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner Author

🔎 Pre-PR Code Review — Reviewer: Pre-PR Code Analyzer

Verdict: APPROVE (pending formal gate) — docs-only change, no code/security/test surface.

Scope reviewed: AGENTS.md + README.md (2 files, +3/−3). The change corrects a false PyPI install claim (pip install datamorph-cli / PyPI badge) to the accurate git+https://github.com/Coding-Dev-Tools/datamorph.git install instruction and a "not published" badge.

Findings:

  • ✅ No logic, security, or dependency changes — pure documentation correction.
  • ✅ CI green (test 3.10–3.13, lint, code-review). ensure-pr fail is the fleet-wide token-scope denial (non-code, not a blocker).
  • ℹ️ The automated review's ruff format warnings reference src/datamorph/*.py and tests/test_*.pynone of those files are in this diff. The warnings are pre-existing repo style drift, not introduced by this PR. No action required here.

Gate note: PR is ~2.5h old and single-author → formal APPROVE withheld per the <6h / <3-contributor embargo. Substantively sound; safe to merge once the gate clears.


Posted by the Hermes Pre-PR Code Reviewer (engraphis-tracked). Reuses prior run verdicts; no new blockers.

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner Author

Council Merge Gate — clean APPROVE, still policy-BLOCKED

  • Run council-merge-gate-20260717T044202Z, session council-e6d4925c-d69c-4f26-b68c-9371233fa1c5 (quorum=4, clean APPROVE).
  • Ground-truth: mergeable=MERGEABLE, required checks green — merge attempted but base-branch policy prohibits it (BLOCKED_BY_BRANCH_PROTECTION).
  • Action required: a human approving review on the PR. Left open; no --admin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

council-approved Council gate approved this PR (APPROVE / APPROVE_WITH_NITS)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant